Skip to content

Conversation

@esope
Copy link
Contributor

@esope esope commented Nov 3, 2025

CHANGES:

  • added support for when clauses

Copy link
Member

@jmid jmid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All green and LGTM, thanks!

Would you consider adding an x-maintenance-intent entry?
https://github.com/ocaml/opam-repository/blob/master/governance/policies/archiving.md

@esope
Copy link
Contributor Author

esope commented Nov 4, 2025

Sure! Done in the latest commit.

@esope
Copy link
Contributor Author

esope commented Nov 4, 2025

The windows/cygwin CI needs to be restarted. There was an error with curl to download the sources of the package, it seems.

@jmid
Copy link
Member

jmid commented Nov 6, 2025

Nit: The combination of merge(?) and force push makes this show up as "4 files changed" in the UI, when in fact there is only 1 🤷 I don't have permission to trigger a rebuild, but you can do so yourself by force pushing a single commit, e.g., on top of a rebase (I admit it is a nit) if you want to see 'all green' 🙂

CHANGES:

- added support for when clauses
@esope esope force-pushed the release-saltoIL-0.2.0 branch 2 times, most recently from 05c60db to 380c992 Compare November 6, 2025 09:10
@esope
Copy link
Contributor Author

esope commented Nov 6, 2025

Well, I'm sorry my git-fu is so bad... Should look better now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants